Skip to content

Conversation

@wsmoses
Copy link
Contributor

@wsmoses wsmoses commented Jan 7, 2025

No description provided.

@wsmoses wsmoses enabled auto-merge (squash) January 7, 2025 07:20
@giordano
Copy link
Member

giordano commented Jan 7, 2025

[07:34:13] In file included from external/llvm-project/llvm/lib/Support/Process.cpp:122:
[07:34:13] external/llvm-project/llvm/lib/Support/Unix/Process.inc:101:19: error: variable has incomplete type 'struct mallinfo'
[07:34:13]   101 |   struct mallinfo mi;
[07:34:13]       |                   ^
[07:34:13] external/llvm-project/llvm/lib/Support/Unix/Process.inc:101:10: note: forward declaration of 'mallinfo'
[07:34:13]   101 |   struct mallinfo mi;
[07:34:13]       |          ^
[07:34:13] external/llvm-project/llvm/lib/Support/Unix/Process.inc:102:10: error: no member named 'mallinfo' in the global namespace
[07:34:13]   102 |   mi = ::mallinfo();
[07:34:13]       |        ~~^
[07:34:13] 2 errors generated.

@wsmoses
Copy link
Contributor Author

wsmoses commented Jan 7, 2025

yeah I tried [and locally] used an updated llvm (llvm/llvm-project#121888 and llvm/llvm-project#121624), but the yggy patches dont apply into it cuz we don't import via xla. I guess we have to wait a few days for google to update its llvm

@wsmoses wsmoses merged commit 8ce34c1 into master Jan 8, 2025
8 of 9 checks passed
@wsmoses wsmoses deleted the wsmoses-patch-1 branch January 8, 2025 07:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants